-
Notifications
You must be signed in to change notification settings - Fork 13
feat(Data/neo4jDatabases): add Neo4j resource type and Kubernetes recipe #58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
feat(Data/neo4jDatabases): add Neo4j resource type and Kubernetes recipe #58
Conversation
Signed-off-by: Nick Beenham <[email protected]>
|
@Reshrahim new PR |
Reshrahim
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @superbeeny. I tested the Recipe deployment and it seems to be working
Suggested minor changes for consistency across definitions in the repo.
| |------------|-----------|--------------------------------------|-------| | ||
| | Kubernetes | Bicep | `kubernetes-neo4j.bicep` | Alpha | | ||
|
|
||
| ## Recipe Input Properties |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For consistency across definitions, would be good to follow this format for Recipe input and output properties -https://github.com/radius-project/resource-types-contrib/blob/main/Data/mySqlDatabases/README.md
zachcasper
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The resource type definition looks good to me. I did not test the recipe. It's difficult to test without a test application.
Signed-off-by: Nick Beenham <[email protected]>
Signed-off-by: Nick Beenham <[email protected]>
Signed-off-by: Nick Beenham <[email protected]>
Description
Replacement PR #45
Related GitHub Issue:
Testing
Contributor Checklist
rad resource-type showis correctenum: []required: []for every object property (not just the top-level properties)readOnly: true